Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a42c4fa) 65.41% compared to head (fe6ddca) 67.60%.

@@            Coverage Diff             @@
##           master      #50      +/-   ##
==========================================
+ Coverage   65.41%   67.60%   +2.19%     
==========================================
  Files           3        3              
  Lines         133      142       +9     
==========================================
+ Hits           87       96       +9     
  Misses         46       46              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@oscardssmith merge?

I thought our BFloat16 plan was to not do anything other than the LLVM intrinsics in Base, and let a package add all the methods, but I might be mistaken here.

EDIT I thought this was in the Julia repo

@milankl Let me know if you'd find it useful to have commit access to this package.

I thought our BFloat16 plan was to not do anything other than the LLVM intrinsics in Base, and let a package add all the methods, but I might be mistaken here.

Shouldn't the long-term plan be to have BFloat16 in Base as are Float16/32/64? But to be less intrusive for now and not do all the work at once, have this package where we/users of BFloat16 can add all missing methods they need for BFloat16? Hopefully we will then eventually reach a maturity here that we can think about moving some of the BFloat16 support into Base? I feel it just requires more general hardware support for more people to actively use BFloat16s.jl till then we can happily add methods here?

@milankl Let me know if you'd find it useful to have commit access to this package.

Yes, I'd be happy to add more methods here whenever I work with BFloat16!